-
Notifications
You must be signed in to change notification settings - Fork 115
[oneDPL] Add set parallel range algorithms #630
New issue
Have a question about this project? Sign up for a free GitHub account to open an issue and contact its maintainers and the community.
By clicking “Sign up for GitHub”, you agree to our terms of service and privacy statement. We’ll occasionally send you account related emails.
Already on GitHub? Sign in to your account
[oneDPL] Add set parallel range algorithms #630
Conversation
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Not finding any issues here, I checked it against the paper, and it all seems good, and not using anything beyond c++20...
LGTM
source/elements/oneDPL/source/parallel_api/parallel_range_api.rst
Outdated
Show resolved
Hide resolved
Co-authored-by: Alexey Kukanov <[email protected]>
std::ranges::set_difference_result<std::ranges::borrowed_iterator_t<R1>, | ||
std::ranges::borrowed_iterator_t<R2>, | ||
std::ranges::borrowed_iterator_t<OutR>> |
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Seems like we missed a mistake here:
std::ranges::borrowed_iterator_t<R2>,
should not be here according to the paper or std::ranges::set_difference_result
definition
There was a problem hiding this comment.
Choose a reason for hiding this comment
The reason will be displayed to describe this comment to others. Learn more.
Good catch. I've prepared #637 to fix it.
Adding
includes
,set_union
,set_intersection
,set_difference
,set_symmetric_difference
parallel range algorithms into oneDPL specification.